Conversation
|
There are some dependency to be cleared. |
### What changes were proposed in this PR? Move dependency `transformer` from `requirements.txt` to `operator-requirements.txt`. ### Any related issues, documentation, discussions? The dependency were introduced #2600 for supporting hugging face operators. It should not have been a dependency for pyamber, but the specific operator. - #2600 This blocks #4088 ### How was this PR tested? Existing tests. ### Was this PR authored or co-authored using generative AI tooling? No
|
waiting for #4091 |
|
blocked by pyiceberg, related issue: apache/iceberg-python#2536 |
|
I want to confirm with @aglinxinyuan. Do you think we can merge this, or do you think we need a vote first? |
There was a problem hiding this comment.
We don’t need a separate vote, since the PR review process already serves as one.
An email is only necessary when soliciting votes on procedural issues or a release, to ensure the majority is aware that a vote is taking place.
For code modifications, the Apache requirement is three +1 votes and no -1 votes for approval. This is why many projects require at least two approving reviews on a PR—combined with the author’s implicit +1, this satisfies the three-vote threshold.
|
sg. Merging now. |
3 participants
What changes were proposed in this PR?
Officially add support for Python 3.13 in Python UDF and native operators.
Any related issues, documentation, discussions?
resolves #4079
How was this PR tested?
New CI.
Was this PR authored or co-authored using generative AI tooling?
No